Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Реализация задачи - Основы: Часть 1 createApp #2

Closed
wants to merge 3 commits into from

Conversation

Mpchest1
Copy link
Contributor

No description provided.

@jsru-1
Copy link
Contributor

jsru-1 commented Sep 27, 2024

Добавляю преподавателя (@ShGKme) для код-ревью.

@jsru-1 jsru-1 requested a review from ShGKme September 27, 2024 22:51
@jsru-1
Copy link
Contributor

jsru-1 commented Sep 29, 2024

Проверьте, пожалуйста, ваше решение, не все тесты прошли (PR не будет принят до тех пор, пока все добавленные задачи не будут решены).

@Mpchest1 Mpchest1 closed this Sep 29, 2024
@ShGKme
Copy link

ShGKme commented Oct 1, 2024

PR - это запрос на вливание одной ветки в другую.

В данном случае на вливание ветки master вашего репозитория (форка) в ваш задачник.
Когда вы добавляете новый коммит в master, он попадает всё в ту же ветку, которая сейчас находится в запросе на слияние, по этому коммит попадает в тот же PR.

По этой причине не нужно закрывать и создавать новый PR при добавлении коммитов в решения (обновлении решения, добавления новых решений и т.д.). Новый созданный PR - точно такой же, как и старый. На слияние той же самой ветки в ту же самую, что и в прошлом PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants